Skip to content

Add function for accessing underlying PKCS#11 keys #102

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

nickrmc83
Copy link
Collaborator

Add a function to allow users of this package to access underlying PKCS#11-isms so that custom extensions can be built. For example the new function could be used to wrap/export a key or unwrap/import a key using the handle associated with an existing private or secret key.

nickrmc83 added 4 commits May 9, 2023 09:37
Add a function to allow users of this package to access underlying PKCS#11-isms so that custom extensions can be built.
For example the new function could be used to wrap/export a key or unwrap/import a key using the handle associated
with an existing private or secret key.
Copy link
Contributor

@solcates solcates left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@solcates solcates left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@phlipse
Copy link

phlipse commented Apr 17, 2024

I'm interested in this PR as well. @solcates seems like you are the only reviewer. Are there any reviewers with write access to the repo left or is it not possible to merge any PR anymore?

Copy link
Contributor

@solcates solcates left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@solcates
Copy link
Contributor

Seems my write access has disappeared since leaving Thales. Anyone else able to help here?

@nickrmc83 nickrmc83 mentioned this pull request Jul 12, 2024
@KenGuan666
Copy link

Hi Nick @nickrmc83 Do you still intend to merge this PR?

I look forward to using the feature you added. This PR needs a rebase though. I'll make a duplicate PR with the same changes if you're no longer working on crypto11.

@IceManGreen IceManGreen self-assigned this Feb 21, 2025
@IceManGreen IceManGreen self-requested a review February 21, 2025 15:23
@IceManGreen
Copy link
Contributor

Working on it

@nickrmc83
Copy link
Collaborator Author

Hi Nick @nickrmc83 Do you still intend to merge this PR?

I look forward to using the feature you added. This PR needs a rebase though. I'll make a duplicate PR with the same changes if you're no longer working on crypto11.

I am unfortunately not a maintainer of this project any more. We'll need someone else to approve the merge.

@IceManGreen
Copy link
Contributor

Hi Nick @nickrmc83 Do you still intend to merge this PR?
I look forward to using the feature you added. This PR needs a rebase though. I'll make a duplicate PR with the same changes if you're no longer working on crypto11.

I am unfortunately not a maintainer of this project any more. We'll need someone else to approve the merge.

Hi @nickrmc83,

Thank you for your reply ! Are you still active on this PR ?
If yes, can you update your branch to follow the Go version of the main remote branch please (v1.23.6) ?
I know this PR is old but still relevant according to @KenGuan666.

@solcates
Copy link
Contributor

solcates commented Mar 1, 2025

I think @nickrmc83 and I are still active users and happy to add PR, but we do not have the right permissions to approve or merge things it seems. We both are former Thales employees, but would be willing to help out a bit with permissions.

@IceManGreen IceManGreen removed the request for review from Nicolas-Peiffer March 3, 2025 09:29
@nickrmc83
Copy link
Collaborator Author

I can look at merging master into this branch. It'll likely be in a couple of weeks though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants